Skip to content

Conversation

@DanielleMaywood
Copy link
Contributor

@DanielleMaywood DanielleMaywood commented Oct 20, 2025

Description

Updates the module to use the new version of the agentapi module for Coder 2.28

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder-labs/modules/cursor-cli
New version: v0.3.0
Breaking change: [x] Yes [ ] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

Related PRs

@DanielleMaywood DanielleMaywood force-pushed the danielle/cursor-cli/refactor branch 2 times, most recently from c524eb5 to ee6aed8 Compare October 21, 2025 08:27
@DanielleMaywood DanielleMaywood force-pushed the danielle/agent-api/refactor branch from 644d080 to 4c9fab8 Compare October 23, 2025 09:14
@DanielleMaywood DanielleMaywood force-pushed the danielle/cursor-cli/refactor branch 2 times, most recently from 6694018 to a0ead99 Compare October 24, 2025 10:23
@DanielleMaywood DanielleMaywood force-pushed the danielle/agent-api/refactor branch from 4c9fab8 to 88773fe Compare October 24, 2025 10:23
@DanielleMaywood DanielleMaywood self-assigned this Oct 24, 2025
Base automatically changed from danielle/agent-api/refactor to main October 24, 2025 10:54
@DanielleMaywood DanielleMaywood force-pushed the danielle/cursor-cli/refactor branch from a0ead99 to f3d707b Compare October 24, 2025 11:04
@DanielleMaywood DanielleMaywood marked this pull request as ready for review October 24, 2025 11:28
@DevelopmentCats
Copy link
Contributor

Right now cursor-cli seems to be erroring out because it looks like cursor has completely removed the -m flag in favor of --model I think this would only require minimal changes which we can probably sneak in here as a minor fix

@DevelopmentCats
Copy link
Contributor

There are a couple things that I will need to address in an issue and a new PR to fix with cursor-cli, but this is at least able to get everything running with coder_tasks and the new provider. @DanielleMaywood

Copy link
Member

@mtojek mtojek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 stamp, but I'll leave the final decision to Chris :)

@DevelopmentCats
Copy link
Contributor

@mtojek @DanielleMaywood

This looks good to me aside from the one example leftover that needs to be changed to main

Overall all of the modules work well with these changes, but some of them have degraded due to recent updates in the agents themselves. I will go ahead and make some issues for these so we can resolve the issues with the other agents, but overall we are good to merge these all I can take care of merging and releasing if yall just wanna commit my last suggestion?

Co-authored-by: DevCats <christofer@coder.com>
@DevelopmentCats DevelopmentCats enabled auto-merge (squash) December 17, 2025 15:11
@DevelopmentCats DevelopmentCats merged commit c551c4d into main Dec 17, 2025
4 checks passed
@DevelopmentCats DevelopmentCats deleted the danielle/cursor-cli/refactor branch December 17, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants